test: add test jack detection if DSP is suspended #1321
Conversation
arikgreen
requested review from
a team,
golowanow,
lgirdwood and
marc-hb
as code owners
arikgreen
force-pushed
the
awilczax/add-test-jack-detection-d3
branch
from
50eabd5 to
e548c1d
Compare
There was a problem hiding this comment.
Pull Request Overview
Adds a new test case to verify jack detection functionality when the DSP is in D3 suspended state. The test uses a USB relay to simulate physical jack plug/unplug events and validates that both jack detection status and DSP power state changes are handled correctly.
- Implements automated jack detection testing using USB relay hardware
- Validates DSP state transitions during jack events while suspended
- Includes comprehensive error handling and configuration options
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| # shellcheck disable=SC1091 source=case-lib/lib.sh | ||
| source "${TESTLIB}/lib.sh" | ||
| # shellcheck disable=SC1091 source=case-lib/relay.sh |
There was a problem hiding this comment.
| # shellcheck disable=SC1091 source=case-lib/relay.sh | |
| # shellcheck source=case-lib/relay.sh |
When someone runs shellcheck outside the top directory, you do want shellcheck to warn them that it cannot find this file.
There was a problem hiding this comment.
I commented only once but I think disable=SC1091 should be removed in both cases.
- Either people run shellcheck -x from the top-level directory as they should. in that case
disable=SC1091is not needed because shellcheck successfully includes the files. - Or, people forget
-xor run shellcheck from a different directory. Then, you want the first warning (out of many other warnings) to be "could not find include files". So, you don't wantdisable=SC1091there either because you don't want to lose the most important warning.
So you never want disable=SC1091
There was a problem hiding this comment.
thanks Marc, I agree
There was a problem hiding this comment.
Afraid disable=SC1091 came back :-(
It's in two other files as well.
arikgreen
force-pushed
the
awilczax/add-test-jack-detection-d3
branch
2 times, most recently
from
d676234 to
fd03264
Compare
arikgreen
force-pushed
the
awilczax/add-test-jack-detection-d3
branch
from
fd03264 to
04418c9
Compare
redzynix
left a comment
redzynix
left a comment
There was a problem hiding this comment.
Some minor changes needed
arikgreen
force-pushed
the
awilczax/add-test-jack-detection-d3
branch
2 times, most recently
from
bfa8930 to
df930dd
Compare
Add a new test case to test jack detection during DSP is in D3 state. The test will check if the jack detection status is updated correctly when the jack is plugged in and unplugged and also if DSP status is changing correctly as well. For unplug/plug headset jack is using a USB relay. https://github.com/darrylb123/usbrelay Signed-off-by: Artur Wilczak <arturx.wilczak@intel.com>
arikgreen
force-pushed
the
awilczax/add-test-jack-detection-d3
branch
from
df930dd to
1a2f2a7
Compare
4 participants
Add a new test case to test jack detection during DSP is in D3 state.
The test will check if the jack detection status is updated correctly when the jack is plugged in and unplugged and also if DSP status is changing correctly as well.
For unplug/plug headset jack is using a USB relay.
https://github.com/darrylb123/usbrelay